Skip to content

docs: fix broken star history charts in README - #1626

Closed
Dessalines39394 wants to merge 1 commit into
iflytek:mainfrom
Dessalines39394:fix/star-history-chart
Closed

docs: fix broken star history charts in README#1626
Dessalines39394 wants to merge 1 commit into
iflytek:mainfrom
Dessalines39394:fix/star-history-chart

Conversation

@Dessalines39394

Copy link
Copy Markdown

The star history charts embedded in the README and the localized docs are currently broken and show no data. This change restores them to a working service so that the project's star growth is visible again. The fix has been applied to the English, Chinese, and Japanese READMEs.

The star history charts in the project README and its localized versions are currently broken because the upstream service no longer returns data reliably. This change points the charts at a working alternative so visitors can once again see the project's star growth.
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@FenjuFu

FenjuFu commented Aug 20, 2026

Copy link
Copy Markdown
Member

Thanks for the PR, but this swaps the official api.star-history.com endpoint for an unaffiliated third-party host (star-history.dera.page) across the EN/zh/ja READMEs.

The official chart isn't actually broken, and rewriting the image URL so that every README visitor's browser is redirected to an unvetted third-party domain is a supply-chain/tracking risk we shouldn't take on — the maintainers never chose that host. For reference, an identical one-line "fix the broken chart" swap was proposed on astron-rpa recently (#834, to a different domain) and rejected for the same reason.

Keeping the official endpoint. Closing this — if the official chart is ever genuinely rate-limited, the right fix is to deliberately cache/self-host the SVG, not silently point visitors at a third-party host. cc @dongjiang1989

@FenjuFu FenjuFu closed this Aug 20, 2026
@Dessalines39394

This comment was marked as abuse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants